Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add TensorKitSectors as a separate module #118

Closed
wants to merge 21 commits into from

Conversation

lkdvos
Copy link
Collaborator

@lkdvos lkdvos commented Apr 22, 2024

No description provided.

lkdvos and others added 19 commits April 21, 2024 15:06
…}` (#1)

* Add scalartype parameter to `AbstractTensorMap`

* Add scalartype in definition of TensorMap

* Add scalartype to AdjointTensorMap

* Add scalartype to BraidingTensor

* Implement changes for indexmanipulations

* Implement scalartype for linalg

* implement changes for tensoroperations

* implement changes for planaroperations

* implement changes for ChainRules

* small fixes

* Fix typo

* Fix missing change

* Formatter

* Apply scalartype changes to tests

* Fix some ambiguities

* De-specialize `trace_permute`

* Despecialize `_contract!`

* Remove unused type-parameter

* Despecialize `planaradd!` and `planartrace!`

* Small fixes

* Change braidingtensor syntax to resolve ambiguities

* despecialize some linalg methods

* formatter

* Add changelog entry

* Fix ambiguity on Julia1.6
This changes the behaviour of copy as an instantiator for creating a TensorMap from a BraidingTensor.
The rationale is that while this does sometimes happen in Julia Base, this is always in the context of lazy wrapper types, for which it makes sense to not copy the parent and then create a new wrapper. BraidingTensor does not wrap anything, so this definition makes less sense.
Incorporate changes from #aada0b2
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

Attention: Patch coverage is 0% with 166 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (370dd92) to head (78620eb).

Files Patch % Lines
src/tensors/tensor.jl 0.00% 43 Missing ⚠️
src/tensors/braidingtensor.jl 0.00% 27 Missing ⚠️
src/tensors/tensoroperations.jl 0.00% 27 Missing ⚠️
src/tensors/indexmanipulations.jl 0.00% 22 Missing ⚠️
src/tensors/abstracttensor.jl 0.00% 18 Missing ⚠️
src/planar/planaroperations.jl 0.00% 14 Missing ⚠️
src/tensors/adjoint.jl 0.00% 7 Missing ⚠️
src/tensors/linalg.jl 0.00% 5 Missing ⚠️
ext/TensorKitChainRulesCoreExt.jl 0.00% 2 Missing ⚠️
src/spaces/productspace.jl 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #118       +/-   ##
==========================================
- Coverage   81.46%   0.00%   -81.47%     
==========================================
  Files          42      36        -6     
  Lines        5588    4757      -831     
==========================================
- Hits         4552       0     -4552     
- Misses       1036    4757     +3721     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lkdvos lkdvos closed this Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants